Skip to content

fix(script): Update set_node_mem.sh to work in MacOS#405

Merged
sharevb merged 1 commit into
sharevb:chore/all-my-stuffsfrom
ra101:fix/update-set_node_mem.sh-for-mac
Jun 25, 2026
Merged

fix(script): Update set_node_mem.sh to work in MacOS#405
sharevb merged 1 commit into
sharevb:chore/all-my-stuffsfrom
ra101:fix/update-set_node_mem.sh-for-mac

Conversation

@ra101

@ra101 ra101 commented Jun 10, 2026

Copy link
Copy Markdown

For MacOS, /proc/meminfo is not a directory, so memory becomes, 0 -1024 which is -1024

❯ echo $OSTYPE
darwin25.0

❯ cat set_node_mem.sh
memory=$(awk '/MemTotal/{print $2}' /proc/meminfo);
memory=$((memory / 1024));
node_options="--max-old-space-size=$((memory - 1024))";
echo "NODE_OPTIONS=$node_options";
export NODE_OPTIONS="$node_options";%                                                                                 

❯ ./set_node_mem.sh
awk: can't open file /proc/meminfo
 source line number 1
NODE_OPTIONS=--max-old-space-size=-1024

❯ git stash pop
## fix/update-set_node_mem.sh-for-mac
 M set_node_mem.sh
Dropped refs/stash@{0} (153c08bc9152c0fc0e43d7fdb77bd35d6033fdb0)

❯ cat set_node_mem.sh
#!/bin/bash

if [[ "$OSTYPE" == "darwin"* ]]; then
    memory=$(sysctl -n hw.memsize)
    memory=$((memory / 1024 / 1024))
else
    memory=$(awk '/MemTotal/{print $2}' /proc/meminfo)
    memory=$((memory / 1024))
fi

node_options="--max-old-space-size=$((memory - 1024))"

echo "NODE_OPTIONS=$node_options"
export NODE_OPTIONS="$node_options"

❯ ./set_node_mem.sh
NODE_OPTIONS=--max-old-space-size=15360

@ra101 ra101 force-pushed the fix/update-set_node_mem.sh-for-mac branch from fb980ed to b623e58 Compare June 10, 2026 12:46
@sharevb sharevb force-pushed the chore/all-my-stuffs branch from 9af41ca to 9bbff46 Compare June 20, 2026 16:46
@sharevb sharevb merged commit 0889c13 into sharevb:chore/all-my-stuffs Jun 25, 2026
5 of 6 checks passed
@sharevb

sharevb commented Jun 25, 2026

Copy link
Copy Markdown
Owner

Hi @ra101 thanks for this PR now merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants